-
Notifications
You must be signed in to change notification settings - Fork 15.4k
[mlir][amdgpu] Add amdgpu.make_dma_descriptor #169407
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[mlir][amdgpu] Add amdgpu.make_dma_descriptor #169407
Conversation
🐧 Linux x64 Test Results
|
|
✅ With the latest revision this PR passed the C/C++ code formatter. |
kuhar
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but wait from a stamp from @krzysz00 before merging
Co-authored-by: Jakub Kuderski <[email protected]>
krzysz00
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM overall
| if (rank != sharedStaticSizes.size()) { | ||
| return emitOpError("tensor must have same rank as tile."); | ||
| } | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should probably check that the atomic barrier is in LDS if it is present (if I'm remembering the conditions correctly)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the review! d51853c I will merge after the tests finish and continue working on the comments on the next one. Will ping you when ready :)
Co-authored-by: Jakub Kuderski <[email protected]>
No description provided.